Feature/refactor test grpc#408
Merged
joshuachp merged 1 commit intoastarte-platform:masterfrom Apr 3, 2025
Merged
Conversation
lucaato
force-pushed
the
feature/refactor-test-grpc
branch
2 times, most recently
from
2faffdb to
1fb3848
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files
🚀 New features to boost your workflow:
|
lucaato
force-pushed
the
feature/refactor-test-grpc
branch
from
1fb3848 to
2998a99
Compare
rgallor
suggested changes
Jan 28, 2025
lucaato
force-pushed
the
feature/refactor-test-grpc
branch
from
2998a99 to
1878298
Compare
lucaato
force-pushed
the
feature/refactor-test-grpc
branch
4 times, most recently
from
a111031 to
fdc7f5d
Compare
lucaato
force-pushed
the
feature/refactor-test-grpc
branch
2 times, most recently
from
64e2526 to
df0890c
Compare
Contributor
Author
|
Requires merging this pr first since a small fix in the message hub proto mock crate is required. EDIT: I'm also going to update the grpc module since the proto got updated. |
lucaato
force-pushed
the
feature/refactor-test-grpc
branch
2 times, most recently
from
bb608bd to
4197d46
Compare
rgallor
approved these changes
Mar 20, 2025
lucaato
force-pushed
the
feature/refactor-test-grpc
branch
4 times, most recently
from
6a61a24 to
039b63e
Compare
lucaato
force-pushed
the
feature/refactor-test-grpc
branch
5 times, most recently
from
3866f4e to
402f2dd
Compare
harlem88
approved these changes
Apr 2, 2025
lucaato
force-pushed
the
feature/refactor-test-grpc
branch
from
402f2dd to
bb4ec7b
Compare
joshuachp
requested changes
Apr 3, 2025
lucaato
force-pushed
the
feature/refactor-test-grpc
branch
from
bb4ec7b to
8a02892
Compare
Reworked to make use of the mock exported in the astarte_message_hub_proto_mock crate. Removed controversial template macro. Signed-off-by: Luca Arato <luca.arato@secomind.com>
lucaato
force-pushed
the
feature/refactor-test-grpc
branch
from
8a02892 to
f0363ae
Compare
joshuachp
approved these changes
Apr 3, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Requires merging this pr first.Requires merging this pr first in the proto definitions repository.Requires merging this #429 pr first.